Skip to content

LTI 1.1 configure number of digits to round score.#3019

Open
somiaj wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
somiaj:lti-1-1-round-score
Open

LTI 1.1 configure number of digits to round score.#3019
somiaj wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
somiaj:lti-1-1-round-score

Conversation

@somiaj

@somiaj somiaj commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

This adds a configuration option $LTI{v1p1}{round_score_digits} which sets the number of digits to round the score to before sending it to the LMS. If this is set to a number less than 2 (nearest whole percent), then rounding of the score is disabled. The default is 2 which is current behavior.

This was requested in #2785.

@drgrice1 drgrice1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@drgrice1

Copy link
Copy Markdown
Member

Actually, it might be a good idea to add this as a variable that can be configured from the LTI tab in the course configuration. So add this variable to those in the @LTIConfigVariables array, and add the appropriate entry in the $LTIConfigValues hash in lib/WeBWorK/ConfigValues.pm.

This adds a configuration option `$LTI{v1p1}{round_score_digits}` which
sets the number of digits to round the score to before sending it to the
LMS. If this is set to a number less than 2 (nearest whole percent), then
rounding of the score is disabled. The default is `2` which is current
behavior. This option was added to the list of possible configurable items
in the LTI course configuration tab.

This was requested in openwebwork#2785.
@somiaj somiaj force-pushed the lti-1-1-round-score branch from 0edb98f to 4068d31 Compare June 16, 2026 15:06
@somiaj

somiaj commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Added this to the list of possibly configurable items on the LTI tab of the course configuration page.

@pstaabp pstaabp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't test, but looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants